Skip to content

Skip flaky Kibana Stats integration test#11807

Merged
ycombinator merged 1 commit intoelastic:masterfrom
ycombinator:mb-skip-flaky-kb-integ-test
Apr 13, 2019
Merged

Skip flaky Kibana Stats integration test#11807
ycombinator merged 1 commit intoelastic:masterfrom
ycombinator:mb-skip-flaky-kb-integ-test

Conversation

@ycombinator
Copy link
Copy Markdown
Contributor

@ycombinator ycombinator commented Apr 13, 2019

We're starting to see errors like this quite frequently (but not all the time) in our CI:

Creating metricbeat91226fb6ffe6d06676cfc1383d0a3ffb19f00db7_kibana_1 ... 
    stats_integration_test.go:60: 
        	Error Trace:	stats_integration_test.go:60
        	Error:      	Should be empty, but was [error making http request: Get http://kibana:5601/api/stats?extended=true: net/http: request canceled (Client.Timeout exceeded while awaiting headers)]
        	Test:       	TestFetch
    stats_integration_test.go:61: 
        	Error Trace:	stats_integration_test.go:61
        	Error:      	Should NOT be empty, but was []
        	Test:       	TestFetch

So this PR skips this test suite for now, while we try to debug and fix the root cause in #11380.

@ycombinator ycombinator added the flaky-test Unstable or unreliable test cases. label Apr 13, 2019
@ycombinator ycombinator requested a review from a team as a code owner April 13, 2019 09:39
@ycombinator ycombinator mentioned this pull request Apr 13, 2019
Closed
@ycombinator ycombinator requested a review from ph April 13, 2019 10:27
@ycombinator ycombinator merged commit 6eecf67 into elastic:master Apr 13, 2019
@ycombinator ycombinator deleted the mb-skip-flaky-kb-integ-test branch April 13, 2019 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cachedout cachedout cachedout approved these changes

@ph ph Awaiting requested review from ph

Assignees

No one assigned

Labels

flaky-test Unstable or unreliable test cases.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ycombinator @cachedout